Skip to content

Conversation

@vogella
Copy link
Contributor

@vogella vogella commented Jan 5, 2026

This PR migrates the usage of the JUnit 4 HeadlessApplicationRule to a new JUnit 5 HeadlessApplicationExtension.

Changes include:

  • Created HeadlessApplicationExtension which implements BeforeEachCallback and AfterEachCallback.
  • Updated HeadlessApplicationElementTest to use @RegisterExtension and JUnit 5 annotations.
  • Updated HeadlessApplicationTest and its subclasses (UIEventTypesTest, UIEventsTest, HeadlessContactsDemoTest, HeadlessPhotoDemoTest, UIStartupTest) to use JUnit 5 imports, annotations, and assertions.
  • Verified that no regressions were introduced by running the tests for org.eclipse.e4.ui.tests.

@vogella vogella force-pushed the junit5-headless-application-extension branch from c2e208a to efdbc5a Compare January 5, 2026 10:08
@github-actions
Copy link
Contributor

github-actions bot commented Jan 5, 2026

Test Results

 3 015 files   3 015 suites   2h 33m 6s ⏱️
 8 258 tests  8 010 ✅ 248 💤 0 ❌
23 598 runs  22 807 ✅ 791 💤 0 ❌

Results for commit d227db1.

♻️ This comment has been updated with latest results.

@vogella vogella force-pushed the junit5-headless-application-extension branch 2 times, most recently from 82e9a15 to 16bde3d Compare January 5, 2026 14:15
@vogella
Copy link
Contributor Author

vogella commented Jan 6, 2026

#3641 tries to fix the flaky test

Created HeadlessApplicationExtension to replace the JUnit 4 HeadlessApplicationRule. Migrated HeadlessApplicationElementTest and its subclasses (HeadlessApplicationTest, UIEventTypesTest, UIEventsTest, UIStartupTest) to use the new extension and JUnit 5 annotations/assertions.
@vogella vogella force-pushed the junit5-headless-application-extension branch from 16bde3d to d227db1 Compare January 6, 2026 17:25
@vogella vogella merged commit 5892b6f into eclipse-platform:master Jan 7, 2026
18 checks passed
@vogella vogella deleted the junit5-headless-application-extension branch January 7, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant